Conversation
KobeArthurScofield
approved these changes
Sep 8, 2026
Member
|
顺便把 vformat 和 vprotogen 前面的 v 给去掉吧，Xray 都改了很多还留这字样也够奇怪的，合了这个 PR 后我更新下 .pb.go |
Member
Author
|
这样不会导致有人的自动化工具爆炸吗 |
Member
|
|
Member
Author
|
有真问题的肯定改没的说 这种一个两个字母的不碍事好端端的我的想法是多一事不如少一事 这样喊的名字可能是怕和其他 format 包重名 |
Member
|
这又没有被 import，重名的也就新 import 的那个包 format，至于 protogen，别的 bin 都不叫这名字 d14767d 暂时不改就不改吧，需要重命名/移动的文件夹有点多比如 XHTTP、RAW 啥的，还想扬掉 pb 配置，以后一起弄吧
|
Member
Author
|
早就想毙了 但是pb杀了api爆炸 这玩意又是百万漕工衣食所系 所以我才很介意加api( |
Member
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#6327 (comment)
这里的想法 直接调用 gofumpt 的函数而非本地二进制 版本完全被 go.mod 控制 避免受到安装 latest 造成的版本差异 而且更干净不涉及本地工具
这个依赖只被独立的命令行工具使用不会被编译进正常 release